-
Notifications
You must be signed in to change notification settings - Fork 850
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
More cleanup in aztables #22453
Merged
Merged
More cleanup in aztables #22453
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Moved constant definitions to constants.go. Moved struct definitions to models.go. Moved internal helper funcs next to their types.
chlowell
approved these changes
Feb 26, 2024
xsuo2022
pushed a commit
to confluentinc/azure-sdk-for-go
that referenced
this pull request
Mar 6, 2024
* Sync eng/common directory with azure-sdk-tools for PR 7385 (Azure#22112) * Update organization of the eng/common/testproxy folder * Add merge-proxy-tags.ps1 and readme to eng/common/testproxy/scripts/tag-merge/ * Extract common assets script functionality to common-asset-functions.ps1 --------- Co-authored-by: Scott Beddall (from Dev Box) <[email protected]> * [Release] sdk/resourcemanager/baremetalinfrastructure/armbaremetalinfrastructure/2.0.0-beta.1 (Azure#22105) * [Release] sdk/resourcemanager/baremetalinfrastructure/armbaremetalinfrastructure/2.0.0-beta.1 generation from spec commit: 90115af9fda46f323e5c42c274f2b376108d1d47 * add removeUnreferencedTypes flag * [azopenai] Updating to the latest 2023-12-01 API (Azure#22081) Generating based on the latest swagger for the 2023-12-01 API: - Chat message types for requests and responses are now different, with fields that are relevant to each more clearly called out. For instance, fields relevant to only assistant type messages (like tools) are separated properly. - Strong types for OYD (ie, bring your own data) scenarios, instead of a `any` parameter. - General updates to just expose new features for models like Dall-E Also, improved the tests a bit to handle the fact that we're often going to several different Azure OpenAI instances for different models, developing features, etc... * azblob: Return io.ErrUnexpectedEOF as error in UploadStream (Azure#22109) * Doc fix (Azure#22117) * Add more retry after headers to check during retries (Azure#22121) * Add more retry after headers to check during retries Include the headers that provide sub-second granularity. * add clarifying comment and negative tests * simplify custom * [azopenai] Fixing some issues with incorrect/incomplete types in generation (Azure#22119) Fixes: - ToolChoice was unmodeled. - ResponseFormat for ChatCompletions wasn't settable using the swagger as we had it (it's an object, not a string) * Prep azcore for release (Azure#22122) * Prep azcore for release * update perf test * Increment package version after release of azcore (Azure#22124) * [azopenai] Updating to the latest azcore to allow for supporting smaller time increments with retry-after-ms (Azure#22125) * azblob: Updating dependencies (Azure#22128) * Add version naming and deprecation link (Azure#21679) * Add version naming and deprecation link * Update MIGRATION_GUIDE.md --------- Co-authored-by: Chenjie Shi <[email protected]> * fix mgmt README.md module link (Azure#22132) * Updating changelog for azblob 1.2.1 release (Azure#22140) * [azopenai] Re-enable the tests for vision and expand it to include Azure. (Azure#22130) - Do proper test recordings for audio so we can enable those. - Bring back vision, and also target Azure. Overall, this brings our test coverage back up to 32% from 24%. Fixes Azure#21598 * Sanitize (Azure#22142) * [Cosmos] Add Global Endpoint Manager (Azure#22100) * gem implementation * missed check on error * not a new variable * Update sdk/data/azcosmos/cosmos_global_endpoint_manager.go Co-authored-by: Matias Quaranta <[email protected]> * Update sdk/data/azcosmos/shared_key_credential.go Co-authored-by: Matias Quaranta <[email protected]> * test cleanup * accept context for i/o bound request * last test case, mutex fixes on lc/gem * Update cosmos_global_endpoint_manager_test.go * cleanup and extending update test --------- Co-authored-by: Matias Quaranta <[email protected]> * Added replaceReadmeModule function (Azure#22135) * Increment package version after release of storage/azblob (Azure#22146) * Prevent panic in exported.Payload (Azure#22148) Treat a nil response body as no body. * Add Snapshot support for SDK (Azure#22118) * remove generated * Make base recording for snapshot tests. Customizable for local testing * comment updates * update test proxy * Changes to Operation-Local header for test Proxy * add comments * add comments * fix regional link * changelog updates * update changelog * update version.go * comment updates * renamed variables/pre-make maps with known sizes for pager items * omit pager from response/options * Add notice for version 1 deprecation (Azure#21680) * Add notice for version 1 deprecation * Update README.md --------- Co-authored-by: Chenjie Shi <[email protected]> * [Release] sdk/resourcemanager/containerservice/armcontainerservice/4.7.0-beta.1 (Azure#22073) * [Release] sdk/resourcemanager/containerservice/armcontainerservice/4.7.0-beta.1 generation from spec commit: b8c74fd80b415fa1ebb6fa787d454694c39e0fd5 * assets --------- Co-authored-by: ReleaseHelper <ReleaseHelper> Co-authored-by: Alancere <[email protected]> * [Release] sdk/resourcemanager/selfhelp/armselfhelp/2.0.0-beta.3 generation from spec commit: b8c74fd80b415fa1ebb6fa787d454694c39e0fd5 (Azure#22077) Co-authored-by: ReleaseHelper <ReleaseHelper> * [Release] sdk/resourcemanager/servicefabric/armservicefabric/2.0.0 (Azure#22079) * [Release] sdk/resourcemanager/servicefabric/armservicefabric/2.0.0 generation from spec commit: b8c74fd80b415fa1ebb6fa787d454694c39e0fd5 * fix live_test --------- Co-authored-by: ReleaseHelper <ReleaseHelper> Co-authored-by: Alancere <[email protected]> * [Release] sdk/resourcemanager/postgresql/armpostgresqlflexibleservers/4.0.0-beta.4 (Azure#22104) * [Release] sdk/resourcemanager/postgresql/armpostgresqlflexibleservers/4.0.0-beta.4 generation from spec commit: 90115af9fda46f323e5c42c274f2b376108d1d47 * Update CHANGELOG.md --------- Co-authored-by: ReleaseHelper <ReleaseHelper> Co-authored-by: Peng Jiahui <[email protected]> * [Release] sdk/resourcemanager/dataprotection/armdataprotection/3.0.0 (Azure#22074) * [Release] sdk/resourcemanager/dataprotection/armdataprotection/3.0.0 generation from spec commit: b8c74fd80b415fa1ebb6fa787d454694c39e0fd5 * fix --------- Co-authored-by: ReleaseHelper <ReleaseHelper> Co-authored-by: Alancere <[email protected]> * [Release] sdk/resourcemanager/healthcareapis/armhealthcareapis/2.0.0 generation from spec commit: b8c74fd80b415fa1ebb6fa787d454694c39e0fd5 (Azure#22076) Co-authored-by: ReleaseHelper <ReleaseHelper> * [Release] sdk/resourcemanager/deviceupdate/armdeviceupdate/1.3.0 generation from spec commit: b8c74fd80b415fa1ebb6fa787d454694c39e0fd5 (Azure#22075) Co-authored-by: ReleaseHelper <ReleaseHelper> * [Release] sdk/resourcemanager/virtualmachineimagebuilder/armvirtualmachineimagebuilder/2.2.0 generation from spec commit: b8c74fd80b415fa1ebb6fa787d454694c39e0fd5 (Azure#22078) Co-authored-by: ReleaseHelper <ReleaseHelper> * [Release] sdk/resourcemanager/network/armnetwork/5.0.0 (Azure#22093) * [Release] sdk/resourcemanager/network/armnetwork/5.0.0 generation from spec commit: 639ecfad68419328658bd4cfe7094af4ce472be2 * fix module version * assets --------- Co-authored-by: ReleaseHelper <ReleaseHelper> Co-authored-by: Alancere <[email protected]> * [Release] sdk/resourcemanager/netapp/armnetapp/6.0.0-beta.1 generation from spec commit: 639ecfad68419328658bd4cfe7094af4ce472be2 (Azure#22092) Co-authored-by: ReleaseHelper <ReleaseHelper> * [Release] sdk/resourcemanager/batch/armbatch/2.2.0 generation from spec commit: 90115af9fda46f323e5c42c274f2b376108d1d47 (Azure#22103) Co-authored-by: ReleaseHelper <ReleaseHelper> * [Release] sdk/resourcemanager/compute/armcompute/5.4.0 (Azure#22114) * [Release] sdk/resourcemanager/compute/armcompute/5.4.0 generation from spec commit: 60679ee3db06e93eb73faa0587fed93ed843d6dc * assets * assets --------- Co-authored-by: ReleaseHelper <ReleaseHelper> Co-authored-by: Alancere <[email protected]> * fix sdk/resourcemanager/connectedvmware/armconnectedvmware README (Azure#22162) * feat: add ReplaceReadmeNewClientName function (Azure#22165) * feat: add ReplaceReadmeNewClientName function * fix * [Release] sdk/resourcemanager/sql/armsql/2.0.0-beta.4 (Azure#22080) * [Release] sdk/resourcemanager/sql/armsql/2.0.0-beta.4 generation from spec commit: b8c74fd80b415fa1ebb6fa787d454694c39e0fd5 * regenerate --------- Co-authored-by: ReleaseHelper <ReleaseHelper> Co-authored-by: Alancere <[email protected]> * sdk/resourcemanager/attestation/armattestation live test (Azure#22143) * sdk/resourcemanager/iotcentral/armiotcentral live test (Azure#22123) * sdk/resourcemanager/labservices/armlabservices live test (Azure#22115) * sdk/resourcemanager/labservices/armlabservices live test * gofmt * sdk/resourcemanager/changeanalysis/armchangeanalysis live test (Azure#22095) * sdk/resourcemanager/msi/armmsi live test (Azure#22094) * sdk/resourcemanager/dnsresolver/armdnsresolver live test (Azure#22091) * sdk/resourcemanager/dns/armdns live test (Azure#22089) * sdk/resourcemanager/quota/armquota live test (Azure#22082) * sdk/resourcemanager/chaos/armchaos live test (Azure#22070) * sdk/resourcemanager/chaos/armchaos live test * fix * sdk/resourcemanager/loadtesting/armloadtesting live test (Azure#22106) * Azdatalake stg 78-82 (Azure#22192) * STG 82 swagger update, azcore, azidentity dependancy updates (Azure#22000) * corrected release date in changelog * STG84 swagger and azcore, azidentity, azblob dependency updates * dependancy updates * indirect dependancy * update code generator version * using STG 82 swagger, and revert code gen version * adding transforms for missing header files * re-generate zz_options.go file * reverting to azblob 1.1.0 * reverting to azblob 1.0.0 * Updated input file from main to latest commit id * Encryption scope SAS (Azure#22099) * corrected release date in changelog * STG84 swagger and azcore, azidentity, azblob dependency updates * dependancy updates * Initial changes for encryption scope sas * indirect dependancy * update code generator version * using STG 82 swagger, and revert code gen version * adding transforms for missing header files * re-generate zz_options.go file * reverting to azblob 1.1.0 * reverting to azblob 1.0.0 * Updated input file from main to latest commit id * added tests for encryption scope sas * add test for user delegation sas * lint error fix * Fix lint error in test * created new datalake encryption scope var * removed blob account encryption scope * push recording * add encryption scope in user delegation * recordings * linter error * minor comments resolve * arm template changes * arm template changes * test fix --------- Co-authored-by: Sourav Gupta <[email protected]> * List system containers (Azure#22108) * corrected release date in changelog * list system containers test * push recording * push recording * recording * recordings * Datalake CPK (Azure#22159) * corrected release date in changelog * datalake cpk * Update CHANGELOG.md add newline changelog * download file & stream tests * recordings * recordings * change to unrecorded test * add negative tests for CPK * [AzDatalake] Set Expiry (Azure#22170) * corrected release date in changelog * setExpiry for files * lint error * recordings * create new sub directory (Azure#22177) * Implement NewSubdirectoryClient * Lowercase d in subDirectoryName Co-authored-by: Gauri Prasad <[email protected]> * Make test recorded and add API calls * Extend testing * Add recorded tests * Update recorded test * doc * Update assets.json * create subdirectory client * changelog * revert to unrecorded test --------- Co-authored-by: Adele Reed <[email protected]> Co-authored-by: adreed-msft <[email protected]> Co-authored-by: Gauri Prasad <[email protected]> * [AzDatalake] Blob swagger update to latest 2021-12-02 (Azure#22178) * blob swagger update to latest 2021-12-02 * Update sdk/storage/azdatalake/internal/generated_blob/autorest.md Co-authored-by: Sourav Gupta <[email protected]> * unmarshall-xml list blobs * unmarshall-xml list blobs --------- Co-authored-by: Sourav Gupta <[email protected]> * service version update * service version upgrade --------- Co-authored-by: Sourav Gupta <[email protected]> Co-authored-by: Adele Reed <[email protected]> Co-authored-by: adreed-msft <[email protected]> Co-authored-by: Gauri Prasad <[email protected]> Co-authored-by: Sourav Gupta <[email protected]> * azidentity credentials preserve MSAL headers (Azure#22098) * [azdatalake] upgrade azblob version to 1.21 (Azure#22196) * upgrade to latest version of azblob with test fixes * upgrade to latest version of azblob with test fixes # Conflicts: # sdk/storage/azdatalake/go.mod # sdk/storage/azdatalake/go.sum * upgrade to latest version of azblob with test fixes * changelog update * removed dead code * recordings * test fix * Various cleanup in azappconfig (Azure#22197) Convert slices of types in-place. Fixed up parameter name for creating snapshots. Correct casing for ETag. Removed duplicate KeyValueFilter type. Add spans for new APIs. * Update package versions (Azure#22201) * Update package versions * upgrade to MSAL v1.2.1 * add token_type to fake tokens * revert debugging --------- Co-authored-by: Charles Lowell <[email protected]> * Increment package version after release of messaging/azeventgrid (Azure#22060) * Increment package version after release of ai/azopenai (Azure#22126) * azblob: STG91 upgrade (Azure#22194) * azfile: STG91 upgrade (Azure#22193) * update source link to new azopenai location (Azure#22208) * link current instead of depricated path * remove newline at end of file * More azappconfig cleanup (Azure#22209) * More azappconfig cleanup Update BeginCreateSnapshot to return the complete Snapshot type. Remove unused response types. Added some missed error checking in a few tests. * fix a few doc comments * Fix spelling (preceeding=> preceding) (Azure#22202) Co-authored-by: Anton Kolesnyk <[email protected]> * Storage: Reject HTTP endpoints (Azure#22183) * [azeventhubs] Fixing an issue where a processor wouldn't grab partitions (Azure#22153) The processor load balancer had a bug where a processor would not claim partitions even if it didn't yet have the proper share. Added in stress tests (balance, multibalance) to run several scenarios with our supported strategies and different parallel owners. Fixes Azure#22097 * Clean up ETags in azappconfig (Azure#22212) Use azcore.ETag instead of string for public surface area. Add doc comments for non-standard ETag option names. * [azopenai] Fixing issue where you can't use whisper with m4a files. (Azure#22210) Fixing issue where you can't use whisper with m4a files. * It's one of the formats that doesn't seem to be recognized without an explicit file extension, which you can pass via Filename * My tests were too heavily dependent on implementation details of the models. Changing this out to check that things are working correctly without checking the exact contents of the response. * Also, rerecorded tests since we're doing multiple audio tests as well. Fixes Azure#22195 * Updating changelog for STG91 preview (Azure#22216) * azfile: STG 91 Preview (Azure#22217) * Increment package version after release of storage/azblob (Azure#22219) * [azopenai] Update changelog for release. (Azure#22215) * Replace string with azcore.ETag (Azure#22218) Missed one in last round of clean-up. * Add script to go mod tidy modules (Azure#22214) Will tidy all modules under the current working directory or under the specified directory. * Increment package version after release of storage/azfile (Azure#22220) * Added Simon and Kushagra as code owners on GO Cosmos SDK (Azure#22221) * [azservicebus] Updating batching to allow for a configurable wait time (Azure#22154) Updating batching to allow for a configurable wait time. Can lead to fuller batches for people that want to tune it. Fixes Azure#19172 * Sync eng/common directory with azure-sdk-tools for PR 7459 (Azure#22222) * check for the presence of a compatible powershell. ensure that we always return a list of tags * allow the script to require pshell6+ * remove the en-us from the link --------- Co-authored-by: Scott Beddall (from Dev Box) <[email protected]> * Fix image tag for multiimage deployments (Azure#22224) Co-authored-by: Liudmila Molkova <[email protected]> * Refresh azcontainerregistry module (Azure#22161) * upgrade to latest codegen * merge resolver * enable span and add changelog * update module name * fix generator tool (Azure#22205) * fix generator tool * gofmt * fix * goimports * azdatalake preview stg82 (Azure#22227) * Rename azidentity/cache pipeline (Azure#22231) * Handle errors in mod tidy script (Azure#22234) * Sync eng/common directory with azure-sdk-tools for PR 7445 (Azure#22233) * Add package filter parameter to select recursively found packages in release * Handle empty value for packageFilter --------- Co-authored-by: Ben Broderick Phillips <[email protected]> * Use package filter to prevent recursive package publish (Azure#22232) * [Cosmos] Add preferred regions to CosmosClientOptions (Azure#22206) * added to client options, added sample * pointers lol * Update example_test.go * Sync eng/common directory with azure-sdk-tools for PR 7451 (Azure#22246) * Added script and pipeline for spec location validation SDK release pipeline would run this validation to ensure the spec comes from the main branch of Azure/azure-rest-api-specs repo * Update parameter * Use github rest api to validate commit * Added token parameter * Support more yaml cases and other languages * Removed the default setting in yaml template * Only validate in case of GA package * Follow APIView to retrieve package version for verification * Get github token from env variable * Removed obsolete parameter --------- Co-authored-by: raychen <[email protected]> * Correct the variable name in spec location validation script (Azure#22247) Co-authored-by: raychen <[email protected]> * Increment package version after release of storage/azdatalake (Azure#22229) * sdk/resourcemanager/storagecache/armstoragecache live test (Azure#22244) * [CODEOWNERS] Remove automation section (Azure#22249) The focus of these changes is to remove the automation section, as CODEOWNERS changes no longer require manual syncing. * Increment package version after release of messaging/azeventhubs (Azure#22251) * Add selective component governance step (Azure#22150) * Add selective component governance step * Use globals.yml in analyze job * Filter packages at verify step (Azure#22252) Co-authored-by: Ben Broderick Phillips <[email protected]> * [azservicebus] Fixing memory leak with go-amqp update. (Azure#22253) go-amqp got a fix recently that replaces its internal tracking map with a counter, which eliminates a rare memory leak we'd see if you were settling a message on a separate receiver _while_ the other receiver was still alive. In recovery scenarios, we discard the old Receiver instance, making this a non-issue for our typical use case. Fixes Azure#22157 * Sync eng/common directory with azure-sdk-tools for PR 7512 (Azure#22241) * generate an artifact with the updated files * bring in Wes's feedback --------- Co-authored-by: Scott Beddall (from Dev Box) <[email protected]> * [Cosmos] Adds global endpoint manager policy and links GEM to client (Azure#22223) * added policy and linked gem to client * refactor go routine for CI pipeline lint * remove debugging print statement * changed to internal pipeline, updated tests * removed unneeded code * saved apiVersion into constant, changed all explicit uses to constant * add gem policy emulator test * small changes to the test * clean up test * pass preferred regions * removed excess comments * add nil check to client options for preferred regions * [azservicebus] Adjust version and changelog, small test change (Azure#22256) * Adjust version and changelog. * Make a test a bit more robust by creating a larger set of messages to pull. * Fix allowed tenants for InteractiveBrowserCredential (Azure#22257) * [azopenai] Implement UnmarshalJSON method for custom types (Azure#22133) A UnmarshalJSON method has been added to the ChatCompletionsToolChoice and ChatRequestUserMessageContent structs. These implementations allow the structs to satisfy the json.Unmarshaller interface, enabling them to decode JSON directly into their respective types. * Increment package version after release of messaging/azservicebus (Azure#22260) * Prepare azidentity v1.6.0-beta.1 for release (Azure#22248) * Increment package version after release of azidentity (Azure#22263) * Update release date for azappconfig (Azure#22259) * Update release date for azappconfig * update to latest azcore * fix LRO options type name * more example cleanup * Sync eng/common directory with azure-sdk-tools for PR 7537 (Azure#22264) * Add ConflictedFile to git-helpers.ps1, add git-helpers.tests.ps1 to exercise basic functionality. * Add `resolve-asset-conflict.ps1` a script that can autoresolve an assets.json file. --------- Co-authored-by: Scott Beddall (from Dev Box) <[email protected]> Co-authored-by: Scott Beddall <[email protected]> Co-authored-by: Ben Broderick Phillips <[email protected]> * Update tooling and sample dependencies (Azure#22267) * Update tooling and sample dependencies Set Go version to 1.18 to prevent CG from scanning go.sum files. This should clean up issues flagged by CG. * skip Go 1.18 from generator build one of its dependencies requires Go 1.19 or later. * add ExcludeGoNMinus2 param to eng job * Increment package version after release of data/azappconfig (Azure#22268) * Update dependencies in deprecated modules (Azure#22269) * Update dependencies in deprecated modules To reduce noise in CG. * fix broken links * Update to latest version of websockets (Azure#22270) CG clean-up. * Add git commit details to stress environment (Azure#22266) Co-authored-by: Ben Broderick Phillips <[email protected]> * Fix Language value for JS (Azure#22275) Co-authored-by: raychen <[email protected]> * Fix typos in ps1 scripts (Azure#22277) Co-authored-by: Lukasz Kokot <[email protected]> * use merge instead of cherry-pick (Azure#22280) Co-authored-by: Scott Beddall (from Dev Box) <[email protected]> * Broaden special case for 403 responses from IMDS (Azure#22278) * Storage/fix Module Name in azfile, azdatalake, azblob (Azure#22273) * fixModuleName adls * fixModuleName blob, file and test case * linter error * test fix * azblob module name change * azblob * azfile lint error fix * comments address * [Release] sdk/resourcemanager/recoveryservices/armrecoveryservicesbackup/4.0.0 (Azure#22107) * [Release] sdk/resourcemanager/recoveryservices/armrecoveryservicesbackup/4.0.0 generation from spec commit: d402f685809d6d08be9c0b45065cadd7d78ab870 * fix * [Release] sdk/resourcemanager/cosmos/armcosmos/2.7.0 (Azure#22184) * [Release] sdk/resourcemanager/cosmos/armcosmos/2.7.0 generation from spec commit: 41e4538ed7bb3ceac3c1322c9455a0812ed110ac * fix module version * assets --------- Co-authored-by: ReleaseHelper <ReleaseHelper> Co-authored-by: Alancere <[email protected]> * [Release] sdk/resourcemanager/hybridcontainerservice/armhybridcontainerservice/1.0.0 generation from spec commit: 41e4538ed7bb3ceac3c1322c9455a0812ed110ac (Azure#22186) Co-authored-by: ReleaseHelper <ReleaseHelper> * [Release] sdk/resourcemanager/containerservice/armcontainerservice/4.7.0 (Azure#22271) * [Release] sdk/resourcemanager/containerservice/armcontainerservice/4.7.0 generation from spec commit: 8b5618d760532b69d6f7434f57172ea52e109c79 * assets * [Release] sdk/resourcemanager/compute/armcompute/5.5.0 (Azure#22188) * [Release] sdk/resourcemanager/compute/armcompute/5.5.0 generation from spec commit: 41e4538ed7bb3ceac3c1322c9455a0812ed110ac * assets --------- Co-authored-by: ReleaseHelper <ReleaseHelper> Co-authored-by: Alancere <[email protected]> * [Release] sdk/resourcemanager/elasticsan/armelasticsan/1.0.0 (Azure#22225) * [Release] sdk/resourcemanager/elasticsan/armelasticsan/1.0.0 generation from spec commit: 907b79c0a6a660826e54dc1f16ea14b831b201d2 * update changelog --------- Co-authored-by: ReleaseHelper <ReleaseHelper> Co-authored-by: Alancere <[email protected]> * [Release] sdk/resourcemanager/streamanalytics/armstreamanalytics/2.0.0-beta.1 (Azure#22198) * [Release] sdk/resourcemanager/streamanalytics/armstreamanalytics/2.0.0-beta.1 generation from spec commit: fa469a1157c33837a46c9bcd524527e94125189a * assets --------- Co-authored-by: ReleaseHelper <ReleaseHelper> Co-authored-by: Alancere <[email protected]> * update changelog (Azure#22289) * Sync eng/common directory with azure-sdk-tools for PR 7580 (Azure#22292) * Support creating resources with user auth * Log warning if TestApplicationId is set * missing space * regenerate md file * Rename * Update link --------- Co-authored-by: jolov <[email protected]> * [Release] sdk/resourcemanager/networkanalytics/armnetworkanalytics/1.0.0 (Azure#22291) * [Release] sdk/resourcemanager/networkanalytics/armnetworkanalytics/1.0.0 generation from spec commit: 21a8d55d74e4425e96d76e5835f52cfc9eb95a22 * update changelog * fix acr test failure (Azure#22290) * Sync eng/common directory with azure-sdk-tools for PR 7569 (Azure#22284) * Correct the name of JS package folder * Uncomment the package verification * Logging more info for troubleshooting * Get sdkType and directory from the package info --------- Co-authored-by: Ray Chen <[email protected]> * Increment package version after release of containers/azcontainerregistry (Azure#22295) * Sync eng/common directory with azure-sdk-tools for PR 7584 (Azure#22298) * Fix role assignment for user auth * PR fb * Apply suggestions from code review Co-authored-by: Heath Stewart <[email protected]> --------- Co-authored-by: jolov <[email protected]> Co-authored-by: JoshLove-msft <[email protected]> Co-authored-by: Heath Stewart <[email protected]> * AzureCLICredential prefers expires_on value (Azure#22299) * [Release] sdk/resourcemanager/hardwaresecuritymodules/armhardwaresecuritymodules/2.0.0-beta.1 (Azure#22185) * [Release] sdk/resourcemanager/hardwaresecuritymodules/armhardwaresecuritymodules/2.0.0-beta.1 generation from spec commit: 41e4538ed7bb3ceac3c1322c9455a0812ed110ac * update directive --------- Co-authored-by: ReleaseHelper <ReleaseHelper> Co-authored-by: Alancere <[email protected]> * [Release] sdk/resourcemanager/appplatform/armappplatform/2.0.0 (Azure#22236) * [Release] sdk/resourcemanager/appplatform/armappplatform/2.0.0 generation from spec commit: 685aad3f33d355c1d9c89d493ee9398865367bd8 * update live test and assets --------- Co-authored-by: ReleaseHelper <ReleaseHelper> Co-authored-by: Alancere <[email protected]> * [Release] sdk/resourcemanager/datafactory/armdatafactory/5.0.0 generation from spec commit: be0d1c383d683a8eb22ab5fd5b75e380ac3c2eea (Azure#22203) Co-authored-by: ReleaseHelper <ReleaseHelper> * [Release] sdk/resourcemanager/cosmos/armcosmos/3.0.0-beta.3 (Azure#22300) * [Release] sdk/resourcemanager/cosmos/armcosmos/3.0.0-beta.3 generation from spec commit: 6f8faf5da91b5b9af5f3512fe609e22e99383d41 * assets * JSON marshaling helpers will preserve Content-Type (Azure#22309) * JSON marshaling helpers will preserve Content-Type The codegen for TypeSpec can explicitly set the Content-Type header. We must prefer this value when set. * fix doc comment * Sync eng/common directory with azure-sdk-tools for PR 7585 (Azure#22312) * Remove package retrieval when verify pkg version * Modified description of PackageName parameter --------- Co-authored-by: Ray Chen <[email protected]> * Go SDK for Azure Web PubSub Data plane (Azure#21929) * generated codespec for web pubsub * Updating readme and autorest * Adding custom client and test code * Add GenerateClientAccessUrl * Resolve comments * resolve comments * Update readme * fix some comments, remove hub parameter when constructing the client, and add HealthAPIClient constructions * Fix go vet error * Fix test failure * Fix comments * Adding recording * Use asset repo * Remove skip * Fix comment * adding main test logic to start the test-proxy * update CI settings * Fix test failure * Update link to temp URL to pass CI * Fix doc comment * Update CHANGELOG.md * resolve comments * Update sdk/messaging/azwebpubsub/ci.yml Co-authored-by: Rick Winter <[email protected]> * Update sdk/messaging/azwebpubsub/sample.env Co-authored-by: Rick Winter <[email protected]> * Resolving comments * resolve comments * remove healthclient * Resolve comments --------- Co-authored-by: MBSolomon <[email protected]> Co-authored-by: Rick Winter <[email protected]> * Added spec location verification to the release pipeline (Azure#22301) * publish package info as artifact * Added the same condition as previous step * [Release] sdk/resourcemanager/springappdiscovery/armspringappdiscovery/0.1.0 (Azure#22187) * [Release] sdk/resourcemanager/springappdiscovery/armspringappdiscovery/0.1.0 generation from spec commit: 41e4538ed7bb3ceac3c1322c9455a0812ed110ac * Update release date --------- Co-authored-by: ReleaseHelper <ReleaseHelper> Co-authored-by: Peng Jiahui <[email protected]> * Replace ErrAuthenticationRequired with AuthenticationRequiredError (Azure#22317) * Sync eng/common directory with azure-sdk-tools for PR 7615 (Azure#22335) * Prepare-Release.ps1: Make dateTime.ToString("MM/dd/yyyy") to work on exotic set-ups On my machine, I experimented with the registry, and the worst part is that I don't remember/know how to reset it back. The work items that script produces, do have datetimes for the upcoming releases in the `MM-dd-yyyy` format, and then I have to correct them by hand. `dateTime.ToString("MM/dd/yyyy")` does produce the date in the format of `MM-dd-yyyy` on my machine. This also happens if I write a corresponding .NET app. The fix that I am proposing makes it work on my specific setup and hopefully breaks no one else. I understand if you are hesitant to take it. Let me know, I'll see how I can restore my setting. But on the other hand, I don't think it makes anything worse, it only makes things more robust, so maybe take it? * Use [CultureInfo]::InvarialtCulture Co-authored-by: Wes Haggard <[email protected]> * Update eng/common/scripts/Prepare-Release.ps1 * Update eng/common/scripts/Prepare-Release.ps1 --------- Co-authored-by: Anton Kolesnyk <[email protected]> Co-authored-by: Wes Haggard <[email protected]> * Fix issue in Verify-Link.ps1 after PS 7.4 update (Azure#22336) https://learn.microsoft.com/en-us/powershell/scripting/whats-new/what-s-new-in-powershell-74?view=powershell-7.4 - Add AllowInsecureRedirect switch to Web cmdlets (Azure#18546) That new option cause a new exception type that exposed a bug where we assumed if InnerException property existed that it was not null. This fix verifies that the property is not null. Co-authored-by: Wes Haggard <[email protected]> * Update packages (Azure#22338) * Prepare internal for release (Azure#22339) * Update package dependencies (Azure#22337) * Update package dependencies * mod tidy on test package * [AzDatalake] Path escape names to file and directory client (Azure#22308) * path escape names to file and directory client * split path and escape logic * recordings * recordings * refactor * add new tests * Prepare azidentity v1.6.0-beta.2 for release (Azure#22341) * Fix BOM issue Generate-DocIndex.ps1 (Azure#22340) Co-authored-by: Daniel Jurek <[email protected]> * Sync eng/common directory with azure-sdk-tools for PR 7634 (Azure#22348) * Fix sparse-checkout git command line behavior * Use command arg parsing behavior in sparse-checkout.yml --------- Co-authored-by: Daniel Jurek <[email protected]> * Sync eng/common directory with azure-sdk-tools for PR 7630 (Azure#22349) * Add ContentType header to Update-GitHubIssue * replaced application/json * Update eng/common/scripts/Invoke-GitHubAPI.ps1 Co-authored-by: Daniel Jurek <[email protected]> --------- Co-authored-by: Peng Jiahui <[email protected]> Co-authored-by: Wes Haggard <[email protected]> Co-authored-by: Daniel Jurek <[email protected]> * Fix azblob/download data race (Azure#22334) * data race while downloadFile * changelog update * cast variable type * Increment package version after release of internal (Azure#22351) * Update to latest internal module (Azure#22149) * Update to latest internal module Add test for ResponseError with nil response body. * go mod tidy * prep for release * Increment package version after release of azidentity (Azure#22352) * Increment package version after release of azcore (Azure#22353) * Update armresourcehealth with latest code generator (Azure#22354) Fixes an issue with parsing ISO8601 time formats. * [azblob] Fix file offset post sequential write in downloadFile (Azure#22357) * fix file offset post download * changelog update * lint error fix * error verifications add * file pointer check in test case * check error from seek function call * [azblob] Fix panic when nil options bag is passed (Azure#22358) * Fix panic when nil options bag is passed * revert variable name change * Cosmos DB: Fixing encoding test (Azure#22359) * Pin azure/login action step to v1.5.1 (Azure#22362) Co-authored-by: James Suplizio <[email protected]> * Add stress test arm/bicep template prefix to New-TestResources ResourceType set (Azure#22365) Co-authored-by: Ben Broderick Phillips <[email protected]> * STG 91 GA [azfile, azblob], azcore version refresh (Azure#22361) * refresh azcore version, module version upgrade * Azfile STG91 GA * azcore version update in azfile * indirect dependancy update * azblob testdata indirect dependancy update * release date change - azblob CHANGELOG.md * release date change - azfile CHANGELOG.md * Update release date - azblob * Update release date azfile * fix flaky test (Azure#22367) * Increment package version after release of storage/azfile (Azure#22374) * Increment package version after release of storage/azblob (Azure#22375) * Revert test change - new file handle open on downloadfile (Azure#22373) * revert new file handle open change on downloadfile * revert new file handle open change on downloadfile * azblob, azcore. azidentity dependancy updates * changelog updates * changelog updates * [key vault] update modules to the 7.5 GA swagger (Azure#22376) * regen * test recordings * update date * update version * fix dates * Increment package version after release of security/keyvault/azsecrets (Azure#22381) * Increment package version after release of security/keyvault/azcertificates (Azure#22380) * Increment package version after release of security/keyvault/azkeys (Azure#22383) * Increment package version after release of security/keyvault/azadmin (Azure#22382) * Fix case-sensitive spelling in scripts that get deployed to C++ repo (Azure#22379) Co-authored-by: Anton Kolesnyk <[email protected]> * changelog update (Azure#22384) * Move emitter-package.json scripts to eng/common (Azure#22387) Co-authored-by: Patrick Hallisey <[email protected]> * [azservicebus] Fixing a memory leak when doing cross receiver settlement. (Azure#22368) Fixing a memory leak when doing cross receiver settlement. go-amqp holds onto some tracking data that won't get cleared if we don't try to settle through the original Receiver. Our fix in here, combined with go-amqp's changes to route to the original receiver, should seal that up. Benchmark added that also doubles as a stress test. Fixes Azure#22318 * [monitor query] move metrics dataplane to new azmetrics module (Azure#22366) * move code * add files * cleanup code * added tests * license * update code coverage * charles feedback * charles feedback * Increment package version after release of monitor/query/azmetrics (Azure#22389) * Update CI to the latest two versions of Go (Azure#22395) * Update CI to the latest two versions of Go * bump to latest golangci-lint * Codeowner update (Azure#22397) * Codeowner update * Update .github/CODEOWNERS * Update .github/CODEOWNERS * Increment package version after release of storage/azdatalake (Azure#22392) * Update GitHubEventProcessorVersion (Azure#22403) Co-authored-by: James Suplizio <[email protected]> * Updates to tools for the CodeownersUtils updates (Azure#22404) Co-authored-by: James Suplizio <[email protected]> * Disable code coverage redesign in CI (Azure#22402) * Disable code coverage redesign in CI Go 1.22 changed some code coverage internals which is causing some cases to under-report coverage. Disabling for now until this is resolved. * conditionally set env var * tweak * move to build matrix * update test regex * Add GOEXPERIMENT to live test matrix * Swap matrix order for display name consistency --------- Co-authored-by: Ben Broderick Phillips <[email protected]> * Fix potential race between NullValue and IsNullValue (Azure#22410) Note that since IsNullValue is on a hot path, it must never block in the common case. * SetBodilessMatcher & SetDefaultMatcher take RecordingOptions (Azure#22405) * upgrade script autorest.go version (Azure#22413) * Storage: Re-enable HTTP endpoints for shared key credential (Azure#22401) * Switch to errors.Join (Azure#22429) * Updates to packages (Azure#22430) * Update packages - aztables - azeventgrid - azqueue * Tidy up the perf tests * Package updates (Azure#22431) * Add logging event EventResponseError (Azure#22411) * Add logging event EventResponseError This event is logged whenever an azcore.ResponseError is created. The contents are from ResponseError.Error(). Add NewResponseErrorWithErrorCode to support creating ResponseErrors when the error code is in a non-standard location. * cleanup * verify no extra logs * Update packages (Azure#22432) * Update azcosmos (Azure#22434) * [azopenai] Adding in an example for gpt-4-vision. (Azure#22439) Fixes Azure#22436 * Update arm internal package versions (Azure#22440) * Updating ARM package dependencies (Azure#22435) * Update github-event-processor version (Azure#22445) Co-authored-by: James Suplizio <[email protected]> * Some cleanup in aztables (Azure#22443) * Some cleanup in aztables Moved options and response types to their respective files. Helper funcs that were used in one place have been removed, their contents made inline. Replaced appending to slices with making the correct size and populating them in place. * fixed up a few more append cases * resource-type added (Azure#22425) * Exposing x-ms-resource-type response header in GetProperties API for file and directory * [azsystemevents] Generation of package for Azure Event Grid system events (Azure#22400) Azure Event Grid system events, now in standalone package form. * More cleanup in aztables (Azure#22453) Moved constant definitions to constants.go. Moved struct definitions to models.go. Moved internal helper funcs next to their types. * [Release] sdk/resourcemanager/apicenter/armapicenter/1.0.0 (Azure#22422) * [Release] sdk/resourcemanager/apicenter/armapicenter/1.0.0 generation from spec commit: d4205894880b989ede35d62d97c8e901ed14fb5a * update * [Release] sdk/resourcemanager/storagecache/armstoragecache/3.4.0-beta.1 (Azure#22419) * [Release] sdk/resourcemanager/storagecache/armstoragecache/3.4.0-beta.1 generation from spec commit: d4205894880b989ede35d62d97c8e901ed14fb5a * update assets * [Release] sdk/resourcemanager/network/armnetwork/5.1.0 (Azure#22418) * [Release] sdk/resourcemanager/network/armnetwork/5.1.0 generation from spec commit: d4205894880b989ede35d62d97c8e901ed14fb5a * update assets * [Release] sdk/resourcemanager/recoveryservices/armrecoveryservicessiterecovery/2.3.0 (Azure#22421) * [Release] sdk/resourcemanager/recoveryservices/armrecoveryservicessiterecovery/2.3.0 generation from spec commit: d4205894880b989ede35d62d97c8e901ed14fb5a * add removeUnreferencedTypes flag * update * [Release] sdk/resourcemanager/alertsmanagement/armalertsmanagement/0.10.0 (Azure#22451) * [Release] sdk/resourcemanager/alertsmanagement/armalertsmanagement/0.10.0 generation from spec commit: 6d2438481021a94793b07b226df06d5f3c61d51d * add removeUnreferencedTypes flag * [Release] sdk/resourcemanager/astro/armastro/0.1.0 (Azure#22414) * [Release] sdk/resourcemanager/astro/armastro/1.0.0 generation from spec commit: d4205894880b989ede35d62d97c8e901ed14fb5a * update to v0.1.0 * [Release] sdk/resourcemanager/largeinstance/armlargeinstance/0.1.0 generation from spec commit: 6051d2b126f5b1e4b623cde8edfa3e25cf730685 (Azure#22427) * Update release date (Azure#22407) * Escape semicolons in query params before parsing (Azure#22460) For compat reasons, url.ParseQuery will reject semicolons within query params. So, we must escape them first. * Update version of github-event-processor (Azure#22461) Co-authored-by: James Suplizio <[email protected]> * Updates to tools for CodeownerUtils updates (Azure#22463) Co-authored-by: James Suplizio <[email protected]> * Add EnforceMaxLifeOfIssues to event-processor.config set to Off (Azure#22465) * Allow relative EmitterPackageJsonPath in New-EmitterPackageLock.ps1 (Azure#22454) Co-authored-by: Patrick Hallisey <[email protected]> * Move to latest azcore (Azure#22462) * [Release] sdk/resourcemanager/redisenterprise/armredisenterprise/2.0.0 (Azure#22423) * [Release] sdk/resourcemanager/redisenterprise/armredisenterprise/2.0.0 generation from spec commit: d4205894880b989ede35d62d97c8e901ed14fb5a * go mod tidy * [Release] sdk/resourcemanager/datafactory/armdatafactory/6.0.0 generation from spec commit: 6051d2b126f5b1e4b623cde8edfa3e25cf730685 (Azure#22428) * azblob: Random write in DownloadFile (Azure#22459) * azblob: Updating changelog for azblob 1.3.1 release (Azure#22467) * Clean up lint in aztables (Azure#22470) Fixed some unused parameters. * Increment package version after release of storage/azblob (Azure#22471) * NewListEntitiesPager fix (Azure#22469) * NewListEntitiesPager fix Per the docs, if there are no continuation header values, there are no more entities to return. * simplify return * Update azappconfig with latest code generator (Azure#22473) * [aznamespaces] Moving to new folder site, updating readme and autorest (Azure#22441) This creates a new module to house the Event Grid Namespaces client that used to be in the root of the `messaging/azeventgrid` module. * azfile, azdatalake: Updating the type of number of chunks to uint64 (Azure#22468) * Add MatchConditions to azcore (Azure#22476) * Updating changelog for v1.2.1 release (Azure#22478) * Updates for aztables metadata (Azure#22472) Include returned metadata in more APIs. Make the amount of returned metadata configurable. * Updating changelog for v1.1.1 release (Azure#22479) * Prep azcore for release (Azure#22481) * Prep azcore for release * fix type-o in comment * Sync eng/common directory with azure-sdk-tools for PR 7682 (Azure#22455) * Support resolving environment variable references in matrix config * Improve type and null handling * Fix reference bug * Change behavior on missing env vars to throw --------- Co-authored-by: Ben Broderick Phillips <[email protected]> * Increment package version after release of azcore (Azure#22488) * Update github-event-processor ver to 1.0.0-dev.20240229.2 (Azure#22489) Co-authored-by: James Suplizio <[email protected]> * Don't error on empty time value (Azure#22485) * Don't error on empty time value Exit early if the value is empty. * improve regex to avoid duplication * [azeventgrid] Standalone package for Event Grid Basic (Azure#22458) Event Grid Basic is the "non-namespace" version of Event Grid. It'll be published in the same package as the other languages under 'messaging/eventgrid/azeventgrid'. * Increment package version after release of storage/azdatalake (Azure#22486) * Increment package version after release of storage/azfile (Azure#22484) * Fix matrix CI tests (Azure#22390) Co-authored-by: Ben Broderick Phillips <[email protected]> * [Cosmos] Add logging changes and sample (Azure#22288) * adds changes to client pipeline and readme sample * fixed padding and improved ReadMe sample * moved headers to cosmos_headers, lowercased values * changes for fully working diagnostics * Update README.md * use latest azcore version * Update README.md * Update cosmos_client_test.go * Fix EnforceMaxLifeOfIssues event and run time (Azure#22513) Co-authored-by: James Suplizio <[email protected]> * [azservicebus, azeventhubs] Updating to take in the new AMQP version to fix issue with sessions taking 26s to close (Azure#22509) Pulling in a new AMQP version with my fix for the "body size < frame size" that would cause to hang for 26 seconds while closing a session on a link that was idle. Fixes Azure#21684 * Sync eng/common directory with azure-sdk-tools for PR 7758 (Azure#22516) * Add 1es changes for job/matrix generation and publish * Use more flexible pool filter for prev/next pool matches * consolidate displayname definition * use linux pool variables for generate matrix job * Fix publish artifact * Use single publish task for publish artifact --------- Co-authored-by: Ben Broderick Phillips <[email protected]> * [azopenai] Updating for the 2024-02-15-preview (Azure#22447) Generating based off the latest preview surface (2024-02-15). TTS is the big feature for this revision and there have been some minor renames for some of the BYOD options. Fixes Azure#22444 * [azblob] Pageblob documentation fix (Azure#22520) * documentation fixes * remove testcommon from examples.go * add random data to page blob instead of empty blob * [azopenaiassistants] Azure OpenAI assistants (Azure#22369) First release of OpenAI assistants - compatible with both OpenAI and Azure OpenAI. * [azeventgrid(s)] Regenerating azsystemevents based on latest commit, updating CHANGELOG dates for release. (Azure#22514) - Prepping all the eventgrid packages for release. All of these releases are betas. - Regenerating the azsystemevents package based on the latest commit. * Sync eng/common directory with azure-sdk-tools for PR 7810 (Azure#22526) * Run publish on failed or succeeded * Expand agent os string detection * Check agent job status env var for artifact name detection * Add sbomEnabled flag to publish template * Fix image and artifact name conditional --------- Co-authored-by: Ben Broderick Phillips <[email protected]> * [azeventgrid(s)] go mod tidy should have happened. (Azure#22527) * Increment package version after release of ai/azopenaiassistants (Azure#22525) * Increment package version after release of ai/azopenai (Azure#22524) * Increment package version after release of messaging/azservicebus (Azure#22521) * Increment package version after release of messaging/azeventhubs (Azure#22522) * Increment package version after release of messaging/eventgrid/aznamespaces (Azure#22528) --------- Co-authored-by: Azure SDK Bot <[email protected]> Co-authored-by: Scott Beddall (from Dev Box) <[email protected]> Co-authored-by: Peng Jiahui <[email protected]> Co-authored-by: Richard Park <[email protected]> Co-authored-by: Sourav Gupta <[email protected]> Co-authored-by: Joel Hendrix <[email protected]> Co-authored-by: David Tesar <[email protected]> Co-authored-by: Chenjie Shi <[email protected]> Co-authored-by: Rick Winter <[email protected]> Co-authored-by: Simon Moreno <[email protected]> Co-authored-by: Matias Quaranta <[email protected]> Co-authored-by: Matthew Solomon <[email protected]> Co-authored-by: Alancere <[email protected]> Co-authored-by: tanyasethi-msft <[email protected]> Co-authored-by: Sourav Gupta <[email protected]> Co-authored-by: Adele Reed <[email protected]> Co-authored-by: adreed-msft <[email protected]> Co-authored-by: Gauri Prasad <[email protected]> Co-authored-by: Honza <[email protected]> Co-authored-by: Charles Lowell <[email protected]> Co-authored-by: Felix <[email protected]> Co-authored-by: Anton Kolesnyk <[email protected]> Co-authored-by: Kushagra Thapar <[email protected]> Co-authored-by: Liudmila Molkova <[email protected]> Co-authored-by: Ben Broderick Phillips <[email protected]> Co-authored-by: raychen <[email protected]> Co-authored-by: Jesse Squire <[email protected]> Co-authored-by: xuanming.zhang <[email protected]> Co-authored-by: Scott Beddall <[email protected]> Co-authored-by: Lukasz Kokot <[email protected]> Co-authored-by: jolov <[email protected]> Co-authored-by: JoshLove-msft <[email protected]> Co-authored-by: Heath Stewart <[email protected]> Co-authored-by: Liangying.Wei <[email protected]> Co-authored-by: Wes Haggard <[email protected]> Co-authored-by: Wes Haggard <[email protected]> Co-authored-by: Daniel Jurek <[email protected]> Co-authored-by: James Suplizio <[email protected]> Co-authored-by: Grace Wilcox <[email protected]> Co-authored-by: Anton Kolesnyk <[email protected]> Co-authored-by: Patrick Hallisey <[email protected]> Co-authored-by: ashruti-msft <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Moved constant definitions to constants.go.
Moved struct definitions to models.go.
Moved internal helper funcs next to their types.